RDKEMW-22317 : Integrate VIPA widget 1.4.4.5 with nativescript - #138

utils/xhr.js:405

• Redirect handling reuses the original request headers and always ends the redirected request without re-sending the original body. This can produce incorrect redirects for non-GET requests (e.g., POST redirects sending Content-Length/Content-Type but no body), and 303 redirects should also switch the stored method to GET for any subsequent redirects.

            var redirectOptions = {
              hostname: redirectUrl.hostname,
              host: redirectUrl.hostname,
              port: redirectPort,
              path: redirectUrl.pathname + (redirectUrl.search || ""),
              protocol: redirectSsl ? "https:" : "http:",
              ssl: redirectSsl,
              method: response.statusCode === 303 ? "GET" : settings.method,
              headers: headers,
              withCredentials: self.withCredentials
            };

src/jsc/modules/lib/URLSearchParams.js:171

• entries() currently returns the internal _list iterator, which exposes the backing tuple arrays to callers. That allows external mutation of tuples without triggering _updateSteps() and can desync URLSearchParams from its associated URL. Consider returning an iterator that yields copied [name, value] arrays instead.

  entries() {
    return this._list[Symbol.iterator]();
  }

src/jsc/modules/lib/URLSearchParams.js:211

• Symbol.iterator currently returns the internal _list iterator. Delegating to entries() keeps iteration semantics consistent across APIs and avoids exposing internal pair arrays.

  [Symbol.iterator]() {
    return this._list[Symbol.iterator]();
  }

utils/xhr.js:11

• The header removed the original MIT license/attribution lines. Since this file is derived from third-party MIT-licensed code, the attribution/license notice should be retained for compliance.

 * Usage: include("XMLHttpRequest.js") and use XMLHttpRequest per W3C specs.
 * Original code by DeFelippi.
 * Some modifications by Comcast.
 *
 */

src/jsc/modules/lib/URLSearchParams.js:171

• entries() currently returns the internal _list array iterator, which exposes the mutable internal pair arrays to callers (mutating an iterated pair would mutate URLSearchParams state). Returning an iterator that yields copied [name, value] pairs better matches platform behavior and avoids unexpected side effects.

This issue also appears on line 209 of the same file.

  entries() {
    return this._list[Symbol.iterator]();
  }